Skip to content

gh-155727: Ignore internal imports in test_attribute_completion - #155730

Merged
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:test-pyrepl-completion-imports
Aug 14, 2026
Merged

gh-155727: Ignore internal imports in test_attribute_completion#155730
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:test-pyrepl-completion-imports

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Aug 13, 2026

Copy link
Copy Markdown
Member

Ignore _pyrepl submodules when comparing the modules imported while the reader runs.

Reader.run_hooks() imports _pyrepl._threading_handler lazily, and the
test counted it as a module imported by the completer.

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The change fix the test as expected.

@serhiy-storchaka
serhiy-storchaka merged commit 94c6066 into python:main Aug 14, 2026
60 checks passed
@miss-islington-app

Copy link
Copy Markdown

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.15.
🐍🍒⛏🤖

@serhiy-storchaka
serhiy-storchaka deleted the test-pyrepl-completion-imports branch August 14, 2026 04:40
@bedevere-app

bedevere-app Bot commented Aug 14, 2026

Copy link
Copy Markdown

GH-155769 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Aug 14, 2026
mbeijen pushed a commit to mbeijen/cpython that referenced this pull request Aug 14, 2026
…pythonGH-155730)

Reader.run_hooks() imports _pyrepl._threading_handler lazily, and the
test counted it as a module imported by the completer.
hugovk pushed a commit that referenced this pull request Aug 27, 2026
GH-155730) (#155769)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants